Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Layered turret top rendering #2769

Closed
wants to merge 6 commits into from

Conversation

CMDR-Bill-Doors
Copy link
Contributor

@CMDR-Bill-Doors CMDR-Bill-Doors commented Sep 21, 2023

Additions

Render turret top in multiple layers, top base -> barrels *n -> top.
If multiple barrel is defined and the gun fires in burst, the barrels reciprocate in alternation.
Everything in a modextension, CombatExtended.TurretDrawExtension

TurretTopGraphicData: graphic data of whatever sprite above the barrels

Barrels: a list of barrels
each barrels consist of:
BarrelOffset: vector2 offset of this barrel
BarrelGraphicData: graphic data of this barrel

TurretBottomGraphicData: graphic data of whatever sprite below the barrels

Reasoning

90mm gun looks stupid with its gunshield reciprocating along with barrel.

Alternatives

Suffer.

Testing

Check tests you have performed:

  • Compiles without warnings
  • Game runs without errors
  • (For compatibility patches) ...with and without patched mod loaded
  • Playtested a colony (specify how long)

@CMDR-Bill-Doors CMDR-Bill-Doors requested review from a team as code owners September 21, 2023 13:11
@github-actions
Copy link

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-6262151456.zip

@github-actions github-actions bot added the Download in Comments This PR has a zipfile download available. label Sep 21, 2023
@github-actions
Copy link

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-6262187508.zip

@github-actions
Copy link

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-6626946018.zip

Copy link
Contributor

@N7Huntsman N7Huntsman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works in my testing, though we'll need some additional texture work done on some of our turrets for the animations.

@perkinslr perkinslr assigned perkinslr and unassigned perkinslr Jan 29, 2024
@perkinslr perkinslr marked this pull request as draft January 29, 2024 06:22
@N7Huntsman
Copy link
Contributor

N7Huntsman commented Jun 23, 2024

Mothballing this, but preserving the branch. It broadly works in testing, but in the absence of someone doing the texture work for the various turret elements, it lacks a purpose. Additionally, the standard visual recoil is satisfactory--may revisit this if someone cares to create the textures.

@N7Huntsman N7Huntsman closed this Jun 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Download in Comments This PR has a zipfile download available.
Projects
Status: Testing and Review
Status: Testing and Review
Development

Successfully merging this pull request may close these issues.

3 participants